Migrate to uv & ruff, remove black and isort by skarndev · Pull Request #290 · pybind/pybind11-stubgen

skarndev · 2026-01-30T14:48:14Z

Summary

This PR migrates the project's tooling from pip/black/isort/flake8 to uv/ruff, and renames master → main in CI.

CI changes

uv: replaced pip with uv for all package management in CI; uv venv + uv pip install for virtual environment setup and dependency installation
ruff: replaced black (formatter) + isort (import sorter) + flake8 (linter) with a single ruff invocation; added a new check job that runs ruff format --diff, ruff check --select I,RUF022 --diff (isort), and ruff check
Branch rename: updated CI trigger and publish step to target main instead of master

Script changes (`tests/`)

install-demo-module.sh: replaced pip install with uv pip install --python <exe> to install into the active venv without requiring pip to be present (uv venv doesn't include pip by default)
check-demo-stubs-generation.sh: replaced black + isort with ruff format + ruff check --select I,RUF022 --fix; added dynamic --target-version to handle PEP 695 syntax in Python 3.12+ stubs; removed --diff flag from the fix step (diffs are shown later by git diff --exit-code)
check-demo-errors-generation.sh: removed uv run wrapper (not compatible with tox's virtual env activation)

tox changes

Removed requirements-dev.txt dependency (replaced with inline deps in tox.ini)
Fixed parallel run: PYTHON_TAG_FILE now uses {envtmpdir} (per-env) instead of a shared path under {toxinidir}/tmp/
Added cmeel-eigen to deps so it's available during commands_pre (cmake build)

Reference stubs

Reformatted all reference stubs under tests/stubs/ with ruff (import ordering and style differ slightly from isort+black)
Updated requirements.txt files: removed black and isort, added ruff>=0.14.14

pyproject.toml

Added exclude = ["tests/stubs"] to [tool.ruff] to prevent ruff from parsing PEP 695 stub syntax (e.g. def f[T](...)) with a py310 target version

🤖 Generated with Claude Code

…ters

- Exclude tests/stubs/ from Ruff: generated .pyi fixtures use Python 3.12+ PEP 695 syntax which is invalid under target-version = "py310" - Run ruff format to fix parse.py after merging main Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

- Replace black and isort with ruff in test requirements.txt files - Fix format_stubs() path (was ./pybind11_stubgen, should be .) and remove --diff flag from ruff check so fixes are actually written - Update reference stubs with ruff's import ordering Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

- Drop uv run from test scripts; use tools directly from active env so both tox and CI work (uv run ignores VIRTUAL_ENV set by tox) - Pass dynamic --target-version to ruff so PEP 695 stubs (py3.12+) are parsed correctly during isort check - Activate .venv in CI before running stub/error check scripts - Update tox.ini: replace requirements-dev.txt deps with cmeel + cmeel-eigen, install local package instead of PyPI pybind11_stubgen - Update reference stubs to match ruff formatting (no blank lines after docstrings, union types without outer parens) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

uv venv does not include pip by default, so plain `pip install` falls through to the system pip and installs demo outside the venv. Using ${PYTHON_EXECUTABLE} -m pip ensures the venv's Python is used. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

uv venv creates environments without pip, so neither `pip install` nor `python -m pip` work. Use `uv pip install --python PYTHON_EXECUTABLE` which explicitly targets the correct interpreter in both CI and tox. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

- Remove blank lines after docstrings (ruff formatting) - Remove accept_annotated_callable from pybind11-v2.11 stubs (function not present in that version) - Update iterator/container type stubs for older pybind11 versions Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

sizmailov · 2026-04-04T15:54:45Z

+      - name: Install uv
+        run: |
+          curl -LsSf https://astral.sh/uv/install.sh | sh
+          echo "$HOME/.cargo/bin" >> $GITHUB_PATH


Any reason to not use a gh action?

- name: Install uv uses: astral-sh/setup-uv@v7

sizmailov · 2026-04-04T16:00:40Z

Probably requirements.txt should not survive migration to uv

Those are dependencies for test stubs, should we consider supplying a pyproject.toml there instead? Or an optional dependency group in the main one?

I don't have a strong opinion on that one. Fine either way with me.

In general keeping number of formats/conventions/required tools makes maintenance simpler.
Sounds aligned with the goal of this PR: reducing friction in devs processes.

(Feel free to dismiss my sporadic comments. As I no longer use the project my perception of right/wrong might be quite a bit off.)

I went with the [dev] section in the main pyproject.toml

…ters

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

- Replace curl uv installation with astral-sh/setup-uv@v7 in CI - Remove per-Python requirements.txt files in favor of pyproject.toml dev group - Add cmeel-eigen to dev dependency group - Move test deps (numpy, scipy, ruff, typing_extensions) into tox deps Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

ax3l · 2026-04-20T21:13:32Z

      - name: Setup annotations on Linux
        if: runner.os == 'Linux'
-        run: python -m pip install pytest-github-actions-annotate-failures
+        run: uv pip install pytest-github-actions-annotate-failures


Codex spotted this:
This now installs pytest-github-actions-annotate-failures before uv sync. uv sync is exact by default, so it can remove packages that are not part of the synced dependency set, which makes that install step effectively redundant unless the package is moved after sync or added to the dev group.

In the current workflow that doesn’t change behavior, because these jobs never run pytest, so nothing consumes that plugin.

skarndev added 12 commits January 30, 2026 17:46

update to uv and ruff

cd8ee31

add missing files

6cfdaaa

install uv + introduce job depedency of build on check

0c85957

fix CI runs

37c4852

update uv.lock

d8a4d26

do not install to venv

a34ce6b

test CI

781a8bf

formatting changes

5c5cb02

create uv venv for stub action

cc18287

swap argument order

107a779

update CI.yml

5a90ba0

update ci

9ab54ac

ax3l mentioned this pull request Apr 3, 2026

Roadmap #286

Open

18 tasks

Sergey Shumakov and others added 8 commits April 4, 2026 08:59

Use uv run for running pybind11-stubgen in CI

d235da4

Merge remote-tracking branch 'origin/main' into feature/modernize-lin…

9595fec

…ters

skarndev marked this pull request as ready for review April 4, 2026 08:00

skarndev requested review from ax3l and virtuald April 4, 2026 08:01

Enabled tox-paralllel runs

e299783

sizmailov reviewed Apr 4, 2026

View reviewed changes

Sergey Shumakov and others added 4 commits April 20, 2026 22:35

Merge remote-tracking branch 'origin/main' into feature/modernize-lin…

6834ee7

…ters

Update reference stubs after merging main

44e5096

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Update pybind11 v2.9 reference stubs and bump cmake to 3.31

9dad310

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

skarndev added the enhancement New feature or request label Apr 20, 2026

ax3l reviewed Apr 20, 2026

View reviewed changes

ax3l approved these changes Apr 20, 2026

View reviewed changes

ax3l merged commit dcdee05 into main Apr 20, 2026
20 checks passed

ax3l deleted the feature/modernize-linters branch April 20, 2026 21:15

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Migrate to uv & ruff, remove black and isort#290

Migrate to uv & ruff, remove black and isort#290
ax3l merged 25 commits intomainfrom
feature/modernize-linters

skarndev commented Jan 30, 2026 •

edited

Loading

Uh oh!

sizmailov Apr 4, 2026

Uh oh!

skarndev Apr 20, 2026

Uh oh!

sizmailov Apr 4, 2026

Uh oh!

skarndev Apr 4, 2026

Uh oh!

sizmailov Apr 5, 2026

Uh oh!

skarndev Apr 20, 2026

Uh oh!

ax3l Apr 20, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

skarndev commented Jan 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

CI changes

Script changes (tests/)

tox changes

Reference stubs

pyproject.toml

Uh oh!

sizmailov Apr 4, 2026

Choose a reason for hiding this comment

Uh oh!

skarndev Apr 20, 2026

Choose a reason for hiding this comment

Uh oh!

sizmailov Apr 4, 2026

Choose a reason for hiding this comment

Uh oh!

skarndev Apr 4, 2026

Choose a reason for hiding this comment

Uh oh!

sizmailov Apr 5, 2026

Choose a reason for hiding this comment

Uh oh!

skarndev Apr 20, 2026

Choose a reason for hiding this comment

Uh oh!

ax3l Apr 20, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

skarndev commented Jan 30, 2026 •

edited

Loading

Script changes (`tests/`)